Skip to content

Typesetting adjustments#213

Merged
conrad-watt merged 5 commits intoWebAssembly:upstream-rebuildfrom
ioannad:typesetting-adjustments
Sep 28, 2023
Merged

Typesetting adjustments#213
conrad-watt merged 5 commits intoWebAssembly:upstream-rebuildfrom
ioannad:typesetting-adjustments

Conversation

@ioannad
Copy link
Copy Markdown
Contributor

@ioannad ioannad commented Sep 15, 2023

See commit messages for details on the changes

- core/exec/instructions.rst:
  + (notify) Error in html: 'You can't use 'macro parameter character #' in math mode
  + (wait) too long right side of formal rules
- core/exec/relaxed.rst (preliminary defs, nit picks):
  + add brackets due to ambiguous '=' signs in predicate definitions of overlapact, sameact, syncact
  + add linebreaks in some too long lines in rangeact, tearfreeact, \X{func}, syncact,
  + add brackets around 'otherwise' and '\iff ...', to match rest of spec
  + add several '~' to add missing space, after all '\iff'
  + add potentially forgotten '&&' before plain '\\'s
- core/exec/relaxed.rst (consistency, nit picks):
  + add several '~' to add missing space, after predicates consistentwith, suspensionsconsistentwith, readsfrom, valueconsistentwith, hbconsistentwith, sclastvisible, notear,
  + removed potentially misplaced '\,'
  + removed and added some arrays to make the lines less long
In relaxed.rst (traces): was creating the error 'Bracket argument to \\ must be a dimension'

It looks less good in the pdf than in the html though.
The store is involved in these reductions as well, as it appears in the
reduction steps of all other memory instructions which refer to a meminst
(all memory instruction reductions except for wait' and fence).
@conrad-watt conrad-watt merged commit 631684c into WebAssembly:upstream-rebuild Sep 28, 2023
conrad-watt added a commit that referenced this pull request Sep 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants